combobox: Remove unused variables

combobox: Create cell view identically

... in the two places it can be created.

file chooser: Don't crawl recent://

This is unnecessary - all the recent files are in the model already.
It also leads to duplicates, since our duplicate filtering is based
on g_file_equal, which does not consider recent:///blabla with
target-uri=/my/example to be the same as file:///my/example.

widget-factory: Test busy state

Makes it easy to test insensitive state of widgets.i
Also exposes brokenness in the shell busy indicator...

HC: no need for green in scales

HC: fix typo

Expand builder menu documentation

List the supported attributes.

gtk-demo: Move the transparent example under Overlay

gtk-demo: Rename textview demos to "Text View"

Fits better with "Tree View".

gtk-demo: Tweak the markup demo

gtk-demo: Add font features to markup demo

GtkTextView: Support font features

Add a ::font-features attribute to GtkTextTag, and support
font features when inserting Pango markup into a text buffer.

placesview: don't show mount for Computer row

The "Computer" row from places view is an abstract volume
that represents the root of the current partition. As such,
it cannot be mounted or unmounted.

Remove the related item on context menu for Computer row.

fontchooserwidget: use appropriate model for GtkTreeIter

We were previously mixing the model used when filtering with an iter that
has been resolved to the backing model.

This results in both an invalid row index as well as an invalid
iter->stamp.

Fix oversights in templates

Some of our templates were not setting the domain attribute
on the <interface> element.

gtk3-demo: Add a markup demo

HC: make progressbar labels more visible

- use the same colro as regular labels, no
need to be more subtle

https://bugzilla.gnome.org/show_bug.cgi?id=752978

HC: make pushed columns visible

https://bugzilla.gnome.org/show_bug.cgi?id=752977

HC: provide explicit treeview styling

deals with some details like:

- DND stuff
- progressbar
- expanders
- separators

https://bugzilla.gnome.org/show_bug.cgi?id=752975

HC: :checked:hover color fix for menuitems

https://bugzilla.gnome.org/show_bug.cgi?id=752969

HC: make link buttons distinct

https://bugzilla.gnome.org/show_bug.cgi?id=752965

HC: whitespace for sidebar

- make some room around the sidebar icons

https://bugzilla.gnome.org/show_bug.cgi?id=752960

gtk3-demo: Add a scrollbar context menu

Just to demonstrate the capability, not because this is
excellent UI.

gtk3-demo: Add another overlay demo

placesview: simplify code

Since we started adding a persistent Computer item,
all the code related to the view modes became obsolete,
since the view is never empty anymore. So, drop this
dead code and use the plain stack to manage the empty
search results view.

Also, this patch fixed a very annoying keyboard navigation
issue where we couldn't go from the On This Computer to Networks
lists, because they were two separate widgets. Merge the two
lists into a single one, and update headers accordingly.

placesviewrow: hook up finalize

It is needed to hook up the finalize method, since
it's overriden manually.

placessidebar: fix ellipsizing glitch

The places sidebar rows' labels were missing a proper
xalign, which caused the labels to keep shaking while
the sidebar was resized.

Fix that by setting the label's xalign to 0.

placesview: plug some memory leaks

The GFile containing the Computer item was not
properly dereferenced, so plug that leak by both
dereferencing it and adding some reference management
on GtkPlacesViewRow.

placesview: show Computer item

GtkPlacesView widget manages persistent locations,
factoring out GtkPlacesSidebar functionality.

It, however, does not completely shows all sidebar
locations, since Computer is still missing.

Add a Computer item, adjusting some internal behavior
to make that possible.

placesview: don't show network addresses

It is distracting, not relevant and too space
consuming.

Remove the network addresses label.

placesview: fix documentation

The doc for gtk_places_view_get_search_query was being
set twice, while gtk_places_view_set_search_query was
never set.

Fix that by correcting the wrong getter documentation.

placessidebar: fix documentation

Use the same explanation of why GtkPlacesSidebar::show-connect-server
was deprecated and its replacements on getter and setter.

Also, fix a mistakenly deprecated function.

inspector: Show monitor information

This is useful to verify the monitor information returned by
other backends.

inspector: Show wayland display information

Don't call it "X display" when we show useful information under
Wayland as well.

wayland: Return a display name

Make gdk_screen_make_display_name() return a likely correct
name for the Wayland socket we are using.

x11: Make selection handling work across screens

When dealing with selection events, we might see windows from
other screens in the requestor field. The current x11 backend
code fails to wrap these in a foreign GdkWindow, since we
don't have the corresponding GdkScreen anymore. Work around
this by creating such 'foreign screens' on demand. We still
maintain the 1:1 relation between the display and the screen
returned by gdk_display_get_default_screen().

https://bugzilla.gnome.org/show_bug.cgi?id=721398

Update Catalan translation

Plug a memory leak

GtkCssProvider was using the wrong unref function for GtkCssKeyframe
objects. Funnily, this didn't crash at all, the refcount was at the
same place in the struct - it just didn't free all the memory, causing
valgrind to complain.

quartz app menu: add hidpi support for menu icons

Use the new cairo to NSImage converter function
to set the device scale.

Remove the pixbuf converter function as this was
the last user.

quartz dnd: add hidpi support for gtk_drag_set_icon_surface()

Instead of storing the dnd icon as a pixbuf store it
as a cairo surface and transfer the device scale
when converting it to a NSImage. In the pixbuf/stock/named
setters convert to a surface instead (no hidpi support yet)

quartz dnd: fix drag icon position relative to cursor position

The y axis is reversed on osx

quartz: disable system shadows if a GdkWindow shadow is set.

The shadow will be drawn in the wrong place in those cases so all we can
do is disable it. This fixes double shadows drawn around menus, popups
and tooltips.

https://bugzilla.gnome.org/show_bug.cgi?id=734984

placessidebar: show Computer item when not in Other Locations

Previous commits removed from places sidebar the code related
to displaying the Computer item, which should be shown when
the sidebar is not displaying the Other Locations item.

Add back the item when the sidebar is not in Other Locations'
mode.

quartz: Fix building

Broken by 6d04183c5d6 and 6ac07a2e85ea

file chooser: Note when searching remote locations

Inform the user that we won't search recursively when searching
in remote locations.

Avoid more GFile<>uri roundtrips

Make GtkSearchHit carry a GFile instead of an uri. Most of the
search engines already have the object around, and converting
to an uri and back is unnecessary extra work.

search engine: Don't crawl remote locations

This is slow, and causes too much network I/O.
So skip locations that look like they are remote.

Add a helper function for remote locations

Add a helper function that says whether a location should be
considered remote. To determine this, we look at the filesystem
type reported by gvfs, and say 'remote' for sftp, webdav, ftp,
nfs and cifs.

Actually use search results from the model

We didn't connect to the signals, so we never picked up
any search results from the model engine. Good thing, since
it was returning the wrong thing.

Port to GtkSearchHit

GtkSearchEngineModel was still trying to return uris as hits.
This is not working anymore, we are expected to return GtkSearchHit
structs.

Always pass a location into the query

Failure to do so was causing search to start from the home dir
when the current folder is not represented on the sidebar.

Store locations as GFile

It is a bit pointless to have the file chooser get a uri from an
existing GFile to put in the query, only to have some of the search
engines reconstruct a GFile from it.

Remove a redundant assignment

Updated Slovak translation

Updated Greek translation

combobox: Use ui file to construct internal button child

combobox: Create button on init() and destroy in destroy()

The button is the same in both menu and list mode, so there's no need to
destroy and recreate it.

combobox: Add scroll support in list mode

combobox: Remove unused argument

The visible check is always FALSE, so no need to keep the code around.

combobox: Remove the box from the button

... when in menu mode.

Now that we don't have a separator anymore, the box is not needed
anymore.

combobox: Remove the separator

It was a special case and it's not used by themes anymore.

Plus, in the CSS world, if a theme wants a separator, it can just
style the button's border to create one.

Remove a duplicate string

dialog-information was twice in the list of accessible labels
for icon names.

listbox: Fix shift-selection

Selecting a range of rows with Shift-click was accidentally broken
in e6e511eb460d8fcd. Make it work again.

quartz: Implement NO_TARGET drag result

This is necessary to make notebook tab DND work.

https://bugzilla.gnome.org/show_bug.cgi?id=752638

quartz: Check if a drag ends inside a known window

If a drag ends inside a known window, set the dest_window field
in the drag context. This information is needed to implemented
notebook tab dragging.

https://bugzilla.gnome.org/show_bug.cgi?id=752638

GtkMenu: Properly detach menu when the window is detached

Without properly cleaning up GtkMenu private attach state
(GtkMenuAttachData) when the attached widget is freed, we would end up
with an invalid pointer to a freed widget. Trying to detach from that
widget would cause a segmentation fault.

https://bugzilla.gnome.org/show_bug.cgi?id=752761

wayland: Fix initial fullscreen monitor support

-1 means that we have no specific preference for an initial
fullscreen monitor, and -1 is less than the number of monitors,
so we would end up accessing invalid memory. Prevent that.

https://bugzilla.gnome.org/show_bug.cgi?id=752875

Fix a warning

Use standard cursor names

This changes GTK+ to use gdk_cursor_new_from_name() with the
'standard' css names, instead of GdkCursorType.

https://bugzilla.gnome.org/show_bug.cgi?id=652085

win32: Support all css cursor names

Approximate some of the resize cursors with similar cursors.

broadway: fix use-after-free on read errors

Call chain:
 - input_data_cb
   - broadway_server_read_all_input_nonblocking (input)
     - broadway_input_free (input)
       (now input is invalid)
     attempt to use input->active -> use-after-free

Make broadway_server_read_all_input_nonblocking return a boolean, TRUE
if the input was valid, FALSE otherwise. This allows input_data_cb to
detect whether the input was gone or not.

https://bugzilla.gnome.org/show_bug.cgi?id=741685

Fix x11-less build

We don't use pangoft2 at all in libgdk, but we are using it in libgtk
both under X and Wayland.

Don't use libcanberra without X

libcanberra calls x11 backend functions, so we can't use it
when the x11 backend is disabled.

places view: Documentation fixups

Rewrite gtk_places_sidebar_set_show_enter_location docs, add
references to the relevant signals, remove mentions of
GtkPlacesView from the docs, since it is not public at this point.

https://bugzilla.gnome.org/show_bug.cgi?id=752633

Added api to set a window to fullscreen on a given monitor.

https://bugzilla.gnome.org/show_bug.cgi?id=752677

Updated Portuguese translation

Updated POTFILES.skip

gtk-demo: Expand the filter model example

Show normal filtering as well.

widget-factory: hook up callbacks to the tab close buttons

Just to show that buttons in notebook tabs work.

Fix a typo

https://bugzilla.gnome.org/show_bug.cgi?id=752810

GdkEvent: Turn out parameters optional

As pointed out by lazka in IRC, /me clearly out of phase after allow-none.

GdkEvent: Fix introspection annotations on gdk_event_get(_root)_coords

These functions respect NULLs being passed, so the return values are
(nullable).

GtkGesture: minor cleanup

We're not really using the coordinates in that function, we just
double check it's not an unrelated event.

MSVC Builds: Fix Introspection Build

The deprecated and a11y sources of GTK+ were spun off some time ago into
their own Makefile.inc, which is then included by gtk/Makefile.am.  Fix the
generation of file sources for the GTK+ .gir.

gtk/gtkplacessidebar.c: Fix Build on C89

Ensure variables are declared at the top of the block, and remove a ';;',
which C89 compilers don't like.

Updated Portuguese translation

Updated Portuguese translation

file chooser dialog: Better spacing

When the dialog is as narrow as possible, we still want some
space between the entry and the search button, so use margins
around the entry and label instead of box spacing.

file chooser: Fix gtk_file_chooser_set_show_hidden

We should not override an explicitly set show-hidden value
when loading the setting.

https://bugzilla.gnome.org/show_bug.cgi?id=610925

inspector: Rescan toplevels when they change

This ensures that windows appear in the inspectors tree when
they are created, and it prevents GTK_DEBUG=interactive from
coming up with an empty object tree.

https://bugzilla.gnome.org/show_bug.cgi?id=752664

inspector: Add a function to rescan

Currently, we only ever scanned the toplevel list and
widget tree once. That is not enough.

Another fix for unmapped window states

The previous fix was falling into the crack between
realized and mapped - we would apply the state when a window
is just realized, then unset the _initially flag, and then
when the window gets mapped, we'd undo the state. To fix
this, go back to the way things were when these flags were
first introduced.

https://bugzilla.gnome.org/show_bug.cgi?id=752765

GtkWidget: Fix set_font_map docs

A typo and non-matching parameter names.

gdk-win32: Send fake release key event for shift key

Windows does not send any release key event for one of the shift keys
when both shift keys were pressed together. This commit solves
the problem by sending the extra release key event for the shift key
which was released as first, when the other shift key is released.

Other modifiers (e.g. Ctrl, Alt) do not have this problem.

https://bugzilla.gnome.org/show_bug.cgi?id=751721

gtk3-demo: Don't use xalign on GtkCheckButton

quartz: remove maximized state when the window position/size is changed

So a window can be maximized/zoomed again after being moved away from
its maximized position. This makes the zoom button on non-CSD windows
work as before.

quartz: implement maximized/unmaximized

Instead of using the default zoom behaviour use the internal
maximized state for selecting our own zoom target. This makes
zooming work for CSD windows where for some reason the
given default zoom target is the current window frame itself
resulting in a shadowless window of the same size.

While this makes the zoom button behave a bit different as expected
it makes things more consistent with other platforms and fixes CSD
zooming.

quartz: make moving CSD windows right up to the screen's menu bar work.

The default implementation prevents the window + shadow to move above
the menu bar. Change it so only the window is considered.

file chooser: Make keynav in save mode smoother

When using the location popup to complete to a directory name
(with a trailing /), we should not punish the user by making the
Open button insensitive and preventing the Enter key from doing
the expected thing (switching to that directory).

https://bugzilla.gnome.org/show_bug.cgi?id=752708

Remove debug spew